Add Markdown support to basic Text widget (Lit).#998
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the Lit renderer to version 0.8.5, introducing markdown support for the Text widget and a new Context.markdown API for dependency injection. It adds a MarkdownDirective that utilizes unsafeHTML and until for asynchronous rendering, along with corresponding tests and sample updates. Feedback suggests improving the logic for prepending markdown heading symbols to prevent redundant formatting when the input text already contains them.
|
Should have clicked "Submit" days ago; the only change I made was to sync the changelog with the newest v0.9 version. |
|
OK, landing this. I co-opted Jacob's v0.9.0 since it hadn't been published before to integrate this change. Next changes to the lit renderer should probably roll the version forward as normal. |
Description
Adds Markdown support to the basic Text widget of the v0_9 Lit renderer.
(This is a very similar, but separate, implementation to what exists in v0.8 today. Adds a couple of tests to the directive.)
Also modifies the demo:gallery09 to inject a Markdown renderer for testing.
Pre-launch Checklist
If you need help, consider asking for advice on the discussion board.